-
Notifications
You must be signed in to change notification settings - Fork 121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: migrating off of CircleCI #638
Conversation
🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use -- conventional-commit-lint bot |
Arggh. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
"""Installs packages from conda forge, pypi, and locally.""" | ||
|
||
# Install from conda-forge and default conda package repos. | ||
session.conda_install(*conda_forge_packages, channel=["defaults", "conda-forge"]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If both default
and conda-forge
have a package, would we get the version from conda-forge
or default
? IMO, we should prefer conda-forge
since those packages are updated more frequently in general.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good question. I don't know the answer yet. I will look it up and see if there is a way to identify a preference for source.
Not gonna hold up the PR for this though. If a change is needed, will add to a separate PR.
Moves
conda/mamba
installs from thecircle/ci
CI/CD pipeline togithub
.Summary:
circle/ci
related config filesnox
session to install the package in aconda/mamba
virtual environment and run the unit testsnoxfile.py
fromowlbot.py
. due to the excessive amount of processing that needs to happen to thenoxfile.py
we made the choice to manually update thenoxfile.py
owlbot.py
to make some changes tobuild.sh
NOTE:
Fixes #398 🦕